Skip to content

[WIP] tooling: Shift tools to bazel#17992

Closed
phlax wants to merge 2 commits intoenvoyproxy:mainfrom
phlax:bazel-repo-locations
Closed

[WIP] tooling: Shift tools to bazel#17992
phlax wants to merge 2 commits intoenvoyproxy:mainfrom
phlax:bazel-repo-locations

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Sep 5, 2021

Commit Message: tooling: Shift tools to bazel
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Sep 5, 2021
@repokitteh-read-only
Copy link
Copy Markdown

repokitteh-read-only bot commented Sep 5, 2021

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).

🐱

Caused by: #17992 was opened by phlax.

see: more, trace.

@phlax phlax marked this pull request as draft September 5, 2021 17:11
@phlax phlax force-pushed the bazel-repo-locations branch 5 times, most recently from 06d5d6e to 10e8ac7 Compare September 6, 2021 05:56
phlax
phlax commented Sep 6, 2021
View reviewed changes
phlax
phlax commented Sep 6, 2021
View reviewed changes
phlax
phlax commented Sep 6, 2021
View reviewed changes
@phlax phlax force-pushed the bazel-repo-locations branch from 10e8ac7 to 792b54e Compare September 6, 2021 06:32
phlax
phlax commented Sep 6, 2021
View reviewed changes
tools/dependency/release_dates.py Outdated
Copy link
Copy Markdown
Member Author

@phlax phlax Sep 6, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this and just hardcode the module name in bazel

@phlax phlax force-pushed the bazel-repo-locations branch 12 times, most recently from 3840afc to ce0def0 Compare September 10, 2021 18:02
@phlax phlax force-pushed the bazel-repo-locations branch from ce0def0 to 9c12adf Compare September 16, 2021 09:32
@phlax
bazel: Add python data providers
191e862 
Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax
bazel: Update tools
efdaf11 
- shift tools to bazel
- make use of bazel data providers

Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax phlax force-pushed the bazel-repo-locations branch from 9c12adf to efdaf11 Compare September 22, 2021 06:37
@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 22, 2021

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot added the stale stalebot believes this issue/PR has not been touched recently label Oct 22, 2021
@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 29, 2021

This pull request has been automatically closed because it has not had activity in the last 37 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot closed this Oct 29, 2021
@phlax phlax reopened this Oct 29, 2021
@phlax phlax removed the stale stalebot believes this issue/PR has not been touched recently label Oct 29, 2021
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 28, 2021

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot added the stale stalebot believes this issue/PR has not been touched recently label Nov 28, 2021
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 5, 2021

This pull request has been automatically closed because it has not had activity in the last 37 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot closed this Dec 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@agrawroh agrawroh Awaiting requested review from agrawroh agrawroh will be requested when the pull request is marked ready for review agrawroh is a code owner

@jwendell jwendell Awaiting requested review from jwendell jwendell will be requested when the pull request is marked ready for review jwendell is a code owner

Assignees

No one assigned

Labels

deps Approval required for changes to Envoy's external dependencies stale stalebot believes this issue/PR has not been touched recently

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@phlax